-
-
Notifications
You must be signed in to change notification settings - Fork 166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: remember previously used links for migration #933
Conversation
sharunkumar
commented
Nov 14, 2023
•
edited
Loading
edited
Thanks for the PR. I think this makes sense but we should have a slightly more iOS friendly design. I also think that we should disable this feature unless the app is installed, just for privacy reasons. I'm happy to take a look at the iOS styles, soon :) |
…conflict resolution
Hi @sharunkumar! Thanks for your patience. I just had some time to take a look. I refactored a few things:
If you could try it out, and let me know what you think, I would appreciate it and we can finally get this merged. Thanks! |
Looks good! Thanks for the update :D |